Skip to content

Feature/dasgen check#4

Merged
aleksisch merged 296 commits into
masterfrom
feature/dasgen-check
Jun 26, 2025
Merged

Feature/dasgen check#4
aleksisch merged 296 commits into
masterfrom
feature/dasgen-check

Conversation

@aleksisch
Copy link
Copy Markdown
Owner

No description provided.

profelis and others added 30 commits April 14, 2025 20:08
Refactor linting conditions to simplify checks for const variable dec…
…towards_better_linter

handle move initialization
…ltline-exprs-wrapping

Support mult line expressions
Why: to make possible create custom thread local impl as example for fibers or coroutines
There already exist include with typedef FLOAT, so change name from
FLOAT to DAS_FLOAT

https://learn.microsoft.com/en-us/windows/win32/winprog/windows-data-types
Why: to make possible create custom thread local impl as example for fibers or coroutines
Fix compilation error with Microsoft include
Add support for version 2 make syntax and update command-line options
instead we demote expressions
…no-empty-ctor

empty ctor are no longer generated
…fix-fixed_arrays

convert [[ ]] to fixed_array instead of array
@aleksisch aleksisch force-pushed the feature/dasgen-check branch 4 times, most recently from 5ecc59b to fe61403 Compare June 10, 2025 00:39
@aleksisch aleksisch force-pushed the feature/dasgen-check branch 5 times, most recently from 08b8d2b to f809685 Compare June 10, 2025 22:31
@aleksisch aleksisch force-pushed the feature/dasgen-check branch from f809685 to 61e37a3 Compare June 11, 2025 13:12
@aleksisch aleksisch force-pushed the feature/dasgen-check branch from 61e37a3 to 08a98d6 Compare June 11, 2025 15:08
borisbat and others added 7 commits June 11, 2025 18:57
…nore-cr-windows

ignore \r on windows inside macros
also clone initialization for smart_ptr etc is now builtin
track temp strings allocations in debug agent
Use SizeOf AlignOf when possible instead of hardcoded numbers.
@aleksisch aleksisch force-pushed the feature/dasgen-check branch from 08a98d6 to cda7a2a Compare June 13, 2025 17:18
@aleksisch aleksisch force-pushed the feature/dasgen-check branch from cda7a2a to 6a81308 Compare June 13, 2025 17:20
@aleksisch aleksisch merged commit 0d2860d into master Jun 26, 2025
21 checks passed
@aleksisch aleksisch deleted the feature/dasgen-check branch July 15, 2025 13:11
aleksisch pushed a commit that referenced this pull request Apr 29, 2026
Fixes 7 review comments in one push:

* MCP stdio safety (#1, #5): read_import and compile_and_collect both
  print warnings/failures to stdout, which corrupts the JSON-RPC
  protocol stream when called from the MCP server. Add `quiet : bool
  = false` to both helpers (default preserves CLI behavior); MCP
  find_duplicates passes true and surfaces failure counts via the
  returned envelope.

* Test portability (#2, #3): replace hard-coded /tmp/... corpus path in
  test_find_duplicates_corpus_not_found with create_temp_file_result +
  immediate remove() — guaranteed-nonexistent path that works on
  Windows runners.

* Newline-delimited paths (#4): MCP do_find_duplicates now normalizes
  \n to , before parse_file_list, matching the README's
  documented "git diff --name-only | …" pipeline use case.

* Project (.das_project) plumbing (#6): add `project` to the
  find_duplicates schema, dispatch, and handler signature; thread
  through to compile_and_collect, which now takes
  `project : string = ""` and uses make_file_access(project) instead of
  the previous hardcoded "" — matching the convention used by
  compile_check / lint / find_symbol.

* Deterministic compile order (#7): both the MCP tool and CLI main.das
  build a sorted array<string> from `keys(against_files)` and iterate
  that, instead of the unspecified-order table-key iteration. Makes
  per-candidate report ordering and --check exit behavior reproducible
  across runs.

Drive-by: PERF006 fix in resolve_against_files (reserve before push
loop).

All test suites still green: 6987 in tests/, 54 find_dupes, 181 MCP
(includes 5 updated find_duplicates tests for new signature). MCP
end-to-end smoke confirms newline-delimited paths and project
parameter both reach the handler.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants